label: Handle crazy corner cases

With ellipsizing, the ellipsized text can have a smaller height than the
non-ellipsized text. So the wider text is also higher. Example:
  .<big>TEXT</big>
will ellipsize to the small text.

Reported-By: Rico Tzschichholz <ricotz@t-online.de>

Fixed Brazilian Portuguese Translation

Bump GLib dependency to 2.35

And drop deprecated g_type_init() calls.

reftests: Add test for recent fix

This test checks the fix in a46368dede684ded093395ac056eaeecf554330d

reftests: Mark windows as popup

This makes the tests run faster and should always be done.

sizerequest: do not derefence NULL pointers

gtkmenubutton: remove weak pointer when needed

We must make sure to remove the weak pointer when disposing the widget
or when resetting the align widget otherwise glib will try to nullify
invalid memory.

gtkmenubutton: set menu to NULL in dispose.

Prevent menu to be detached multiple times.

sizerequest: Restructure code

Make the compute_size_request() function take into account size groups
itself instead of doing a weird "bump_requisition" call.

sizerequest: Move sizegroups function to different source file

sizegroup: Construct the set of sizegroup peers as a hash table

This way we don't need a marker on GtkWidgetParivate that needs to be
unset later, so we have all our data in the same place and can avoid
problems with reentrancy and shenanigans like that.

But the main reason I wrote that is cleaner code.

reftests: Add a test from evolution

This was used in a bug report.

https://bugzilla.gnome.org/show_bug.cgi?id=677609

reftests: Add a test for all the recent sizegroup hackery

sizegroup: Use _gtk_widget_compute_size_for_orientation()

With this function now available, we can do size computation in 2
ways:
(1) Compute size with size groups
(2) Compute size without size groups

And have (1) use (2) instead of setting flags on widgets. This patch
does exactly that.

sizerequest: Move optimization

With size groups now doing hfw, doing the optimization for CONSTANT_SIZE
was done too early. Size groups need to know that it's a hfw request, so
the other widgets in the size group get the correct behavior.

label: Redo get_preferred_width/height()

This is important for size groups mostly, but also has some small fixes.
The label-sizing reftest as been updated accordingly.

label: Fix ellipsize and wrap being set

The label code assumed that Pango treats this as "wrap to as much space
as possible and then ellipsize all the lines", but for Pango, ellipsize
takes precedence over wrap. So do the same thing in GtkLabel.

Also updated is the reftest that checked this behavior.

reftests: Improve reftest performance

Get rid of all the event boxes in this test. Event boxes need GDK
windows which cost a lot of performance when running the test and they
clip the label output.

Getting rid of the clipping also shows 2 bugs in this test that weren't
visible before. Those will be fixed in a followup patch.

reftests: Make label-sizing tests use better CSS

ow that labels can have backgrounds, just use label backgrounds

sizerequest: Export _gtk_widget_compute_size_for_orientation()

and add an "ignore_size_groups" flag to it. This way we can use it for
size group shenanigans.

sizegroup: Move GtkSizeGroupMode to gtkenums.h

This is in preparation for the next patch, which would otherwise lead to
conflicts.

sizerequest: Cache sizes without size groups

We compute on-demand for size groups anyway, so we can (in theory, this
patch doesn't do that yet) get around costly cache blowing when
invalidating single widgets of a size group this way.

sizegroups: Use is_visible() instead of get_mapped() for visibility

The current approach of using gtk_widget_get_mapped() is broken:
The usual steps taken when showing a window are:
(1) request the sizes
(2) allocate the sizes
(3) show the window in the allocated size

Showing the window with a random size between steps (1) and (2) would of
course
result in extra work and potential flickering when the widgets get
resized to
their proper sizes.

However, as GtkSizeGroup::ignore-hidden uses gtk_widget_get_mapped() to
determine visibility for a widget, the following will happen:
(1) the widget will request a 0 size
(2) the widget will be allocated a 0 size
(3) the widget will be too small when it is shown

gtk_widget_get_visible() however is set in advance. Note that toggling
visibility also causes a gtk-widget_queue_resize() call already so we
take care of changes in here automatically.

API: Add gtk_widget_is_visible()

This is a recursive gtk_widget_get_visible(): Returns TRUE if the widget
and all its parents are visible.

sizegroup: Handle hfw in size groups

sizegroup: Add a function for clarity

... and restructure code to accomodate that function.

sizegroup: Don't keep groups around everywhere

The code is only interested in the actual widgets that belong together,
not in the groups. So just don't return the groups.

sizegroup: Check ignore_hidden flag when adding groups

Instead of only checking the ignore_hidden flag when getting the
preferred sizes, respect it already when constructing the list of
widgets. This way, widgets don't queue resizes for groups they're
ignored in anyway.

sizegroup: Use for loops

For loops to loop over lists look nicer and actually do the right thing
with "break" and "continue" statements. So they are vastly preferred to
while loops.

sizegroups: Restructure code

This way, we do the checks at the start of the effected function, not
before calling it.

sizegroup: Don't cache the sizes anymore

This simplifies code and because sizes are cached by the widgets
themselves, it's not a large performance problem (unless people use huge
amounts of widgets in a single size group, but who does that?

Merge branch 'bgo687196-filesystemmodel-crash'

bgo#687196 - Fix model corruption during file removal

The main problem is that we were emitting the row-deleted signal for the model in the middle
of the process that actually deletes the row from the model (remove the row from the array,
update the model->file_lookup hash table, etc.).  In the model's caller, one of the row-deleted
callbacks was requesting an iter, which caused the model to revalidate itself - but it did
this while it was in an inconsistent state.  This led to an assertion failure later when the
model resorted itself.

The fix in remove_file() is like this:

* The filteredness/visibility of the deleted node is not updated.  The
  node will simply be gone; we don't need to update those values at
  all.

* We invalidate just the node that is being deleted.

* The model->file_lookup hash table is not completely nuked; instead,
  we carefully adjust its indices.

* The row-deleted signal is only emitted at the very end, when
  deletion is complete and the model is consistent.

Many thanks to William Hua for doing the detective work on this bug!

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Make freeze_updates() and thaw_updates() static functions

They were in the semi-public API of GtkFileSystemModel, but never actually used outside of it.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Comments on how the filtering and sorting processes work

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

sizegroup: Always at least use widget's size

When widgets were hidden, they were otherwise assigned a 0 size.

settings: Reset all styles when the enable-animations settings changes

https://bugzilla.gnome.org/show_bug.cgi?id=686021

Raleigh: Fix spinners with disabled animations

When animations are disabled, active and inactive spinners should look
different.

https://bugzilla.gnome.org/show_bug.cgi?id=686021

cssimage: Implement some equal functions

Remove argument to _gtk_file_system_model_update_file() that should not be part of the internal API

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename gtk_tree_path_new_from_node() to tree_path_new_from_node()

This is a function internal to the file system model; let's not pollute the gtk_tree_path namespace.

Also, make the 'i' variable into 'r' as it refers to a row index, not a file-array index (for
consistency with the docs and the rest of the code).

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

icon-theme: support loading symbolic GFileIcons from generic URIs

Right now we support loading and recoloring symbolic GFileIcons, but
only if the underlying GFile has a local path. This breaks when the
GFileIcon is loaded from a GResource, which is a reasonable option for an
application that wants to ship a custom symbolic icon.

This patch changes GtkIconInfo to store a GFile together with the file
path, and changes the symbolic icon lookup code to use the GFile URI,
which transparently makes the code work also for GResources.

https://bugzilla.gnome.org/show_bug.cgi?id=687059

themingbackground: Remove struct members

... and put them in the only function they are used in.

reftests: Add reftest for fractional border sizes

themingengine: Draw fradctional border sizes

reftests: Fix linear-gradient reftest

... and add it to the Makefile

themingbackground: Remove GtkThemingBackgroundLayer

The struct was just the index. So just pass the index around instead of
a full struct.

themingbackground: Use get_box() for background image size

themingbackground: Introduce gtk_theming_background_get_box()

to query the different clip boxes used by the background drawing code.
Use this function to query these boxes when clipping.

themingbackground: Add content_box variable

... to go with border_box and padding_box.

themingbackground: Get rid of flags variable

themingbackground: Move image variable

... from the Layer struct onto the stack of the only function using it.

themingbackground: Restructure code some more

Move variable initialization outside the first code with side effects.
This allows adding some more early returns, including one for code that
used to trigger g_return_if_fail() in certain corner cases.

themingbackground: Restructure code

Make if statements encompassing the whole function into early returns.
The rest of the diff is reindenting.

cssimage: Add a warning for drawing empty images

width and height of an image must be > 0 for the image to get drawn.
It's up to the code further up to ensure that this is not happening.

reftests: Add a reftest for recent commit

Check that a computed background-size of 0 is treated as 0.

cssvalue: Compute "background-size: 0 0" properly

Previously a computed value of 0 was treated as "auto", which is wrong.

stylecontext: Don't use bg image in gtk_style_context_set_background()

Old code tried to use the "background-image" proeprty for setting the
default image background. While this used to work in the early days of
GTK3, today it is grossly misleading as the backgronud image may be
resized, repositioned and semi-translucent which causes very weird
artifacts when rendering.

So we use the background-color only instead.

cssimage: Add an equal vfunc

No implementations for it exist yet.

menuitem: Draw background unconditionally

settings: Use _gtk_css_provider_load_named()

This way we create one provider per settings object instead of stuffing
it into a global unchanging never-deleting hash table.
Also, we now reload the theme when instructed instead of keeping the old
loaded (and possibly stale) data forever.

https://bugzilla.gnome.org/show_bug.cgi?id=683896

cssprovider: Move fallback code into _gtk_css_provider_load_named()

This makes sure the full theme loading logic resides in one function and
isn't scattered around.

As a side-effect, the hash table kept by gtk_css_provider_get_named()
will now be populated with fallback themes. This will not be a problem
after the next commit though.

cssprovider: Export gtk_css_provider_load_named) function

... and document it.
For now, the function is only exported internally.

cssprovider: Split out theme loading function

Split maintaining the global themes hash table and the theme loading
code into two functions.
This also fixes leaking the provider when loading a theme from a builtin
resource.

Updated Galician translations

Updated Marathi Translations

Updated German translation

[l10n] Updated Estonian translation

Updated Slovenian translation

Updated Lithuanian translation

wayland: update to work with stable libxkbcommon

libxkbcommon has had some changes to its API. However, it now has a
stable release (0.2.0), so this makes the necessary changes, and
replaces all uses of the deprecated API.

Signed-off-by: Ran Benita <ran234@gmail.com>

Updated Spanish translation

Updated Indonesian translation

Updated gujarati file

Updated Serbian translation

Fix compiling for win32

https://bugzilla.gnome.org/show_bug.cgi?id=687066

texthandles: Add an extra style class to the cursor-mode handle

Themes may want to render handles differently depending on whether
the widget is in selection mode (2 handles enclosing a selection) or
cursor mode (one handle pointing out the insertion cursor).

texthandles: set input shape on handles' window

This improves both interaction and theming, as it allows
arbitrary handle shapes while just being draggable from
the visible areas.

This way themes can set up handles with the hotspot visually
displaced from the horizontal center, as long as the hotspot
lies centered in the image/svg asset.

texthandle: Fix shape setup on non-composited environments

The check on the handle to be drawn on the mask was based on the yet to
be set priv->windows pointers, pass explicitly the handle position to
have the shape correctly initialized on non-composited environments

Assamese translation updatedas.po

Fix typo from f2ab3af

Updated Latvian translation

configure.ac: Look for wayland-client 1.0.0

Add GDK_VERSION_3_8

This macro is needed for the new AVAILABLE_IN and DEPRECATED_IN
macros.

Updated Greek translation

Bump version

3.7.0

Be robust against unrealized windows in GtkWindowAccessible

Based on a patch by Albert Astals Cid,
https://bugzilla.gnome.org/show_bug.cgi?id=686152

Forgotten fixup

This was meant to be included in the previous commit :-(

treeview: Add support for styling the dragged header

https://bugzilla.gnome.org/show_bug.cgi?id=684980

treeview: Move the dragged header in the headers window

https://bugzilla.gnome.org/show_bug.cgi?id=684980

Use named union for _GtkSymbolicColor in gtk/gtksymboliccolor.c

Patch by Richard Lloyd,
https://bugzilla.gnome.org/show_bug.cgi?id=686366

scrollbar: remove unused variable

My bad.

notebook: return TRUE for drag-motion event when over tabs

The GtkNotebook drag-motion event handler may install a timeout when
hovering over a tab, in order to switch to it.
On the other hand it's desirable for applications to use the empty tab
area as a drop target, so the drag-motion handler returns FALSE
(also in case it installs the switch tab timeout), as explained in
https://bugzilla.gnome.org/show_bug.cgi?id=350665.

Unfortunately, applications can use the tab label widget (or a child
of it) as a different drop target area, and install their own
drag-motion handler there.
In this scenario, the timeout will still be installed by GtkNotebook's
handler, but since it returns FALSE, it will never get the matching
drag-leave event, causing it to trigger also when the mouse pointer
moved elsewhere before it expired.

Fix this by returning TRUE from drag-motion when the event is over a
tab. Note that this makes automatic tab switching not work anymore when
drag and drop is handled in the tab label widget; applications are
expected to also handle tab switching if desired in such a case.

https://bugzilla.gnome.org/show_bug.cgi?id=684415

notebook: consolidate code to remove the switch tab timer

https://bugzilla.gnome.org/show_bug.cgi?id=684415

[l10n] Updated Estonian translation

scale: update style properties on GtkRange at init

Same fix as in ef027c93d49e2e31ccb8c181d0096c705c141906, but for
the GtkScale subclass of GtkRange.

https://bugzilla.gnome.org/show_bug.cgi?id=686280

scrollbar: update style properties on GtkRange at init

GtkScrollbar used to rely on style-updated being emitted every time
after the widget was created in order to set the right values from its
style properties on GtkRange.
Nowadays we try to be smarter and avoid emitting style-updated at
creation time, so we need to manually initialize the GtkRange values.

This fixes a regression from 35e36b9fe506143f722a00f26a86143f9c95b740.

https://bugzilla.gnome.org/show_bug.cgi?id=686280

Fix out of source tree building for gdk/broadway

This patch fix compilation when building out of source tree (gdk/broadway dir).

Signed-off-by: Colin Walters <walters@verbum.org>

Updated Slovak translation